Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3.2.7 #578

Closed
wants to merge 582 commits into from
Closed

3.2.7 #578

wants to merge 582 commits into from

Conversation

dorshay6
Copy link
Contributor

ℹ️ Change description:

Nothing here

templates

[templates: bug, feature]

@lgtm-com
Copy link

lgtm-com bot commented Oct 29, 2022

This pull request introduces 15 alerts when merging 4ae94b6 into 2374de1 - view on LGTM.com

new alerts:

  • 13 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

@lgtm-com
Copy link

lgtm-com bot commented Oct 31, 2022

This pull request introduces 15 alerts when merging a2e6dcf into 2374de1 - view on LGTM.com

new alerts:

  • 13 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

@lgtm-com
Copy link

lgtm-com bot commented Oct 31, 2022

This pull request introduces 20 alerts and fixes 5 when merging c42a118 into 2374de1 - view on LGTM.com

new alerts:

  • 18 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

fixed alerts:

  • 5 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 1, 2022

This pull request introduces 21 alerts and fixes 5 when merging 47f633c into 2374de1 - view on LGTM.com

new alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

fixed alerts:

  • 5 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 6, 2022

This pull request introduces 21 alerts and fixes 5 when merging f7b5853 into 2374de1 - view on LGTM.com

new alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

fixed alerts:

  • 5 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 6, 2022

This pull request introduces 21 alerts and fixes 5 when merging 768dd92 into 2374de1 - view on LGTM.com

new alerts:

  • 19 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

fixed alerts:

  • 5 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 8, 2022

This pull request introduces 22 alerts and fixes 5 when merging f084309 into 2374de1 - view on LGTM.com

new alerts:

  • 20 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

fixed alerts:

  • 5 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 10, 2022

This pull request introduces 22 alerts and fixes 5 when merging 73ffea3 into 2374de1 - view on LGTM.com

new alerts:

  • 20 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

fixed alerts:

  • 5 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 10, 2022

This pull request introduces 22 alerts and fixes 5 when merging 50ebacc into 2374de1 - view on LGTM.com

new alerts:

  • 20 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

fixed alerts:

  • 5 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 16, 2022

This pull request introduces 22 alerts and fixes 5 when merging 4c27e20 into 2374de1 - view on LGTM.com

new alerts:

  • 20 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

fixed alerts:

  • 5 for Unused variable, import, function or class

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link

lgtm-com bot commented Nov 17, 2022

This pull request introduces 27 alerts and fixes 9 when merging c154485 into 2374de1 - view on LGTM.com

new alerts:

  • 25 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

fixed alerts:

  • 9 for Unused variable, import, function or class

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link

lgtm-com bot commented Nov 21, 2022

This pull request introduces 27 alerts and fixes 9 when merging c7588cb into 2374de1 - view on LGTM.com

new alerts:

  • 25 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

fixed alerts:

  • 9 for Unused variable, import, function or class

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link

lgtm-com bot commented Nov 23, 2022

This pull request introduces 27 alerts and fixes 9 when merging 76c621e into 2374de1 - view on LGTM.com

new alerts:

  • 25 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

fixed alerts:

  • 9 for Unused variable, import, function or class

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link

lgtm-com bot commented Dec 7, 2022

This pull request introduces 29 alerts and fixes 10 when merging 2296022 into 2374de1 - view on LGTM.com

new alerts:

  • 27 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Superfluous trailing arguments

fixed alerts:

  • 10 for Unused variable, import, function or class

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

OmerGery and others added 29 commits October 9, 2023 18:12
* start support offline method

* wip

* wip

* wip

* fixes

* Update index.tsx

* wip

* Update index.js

* change setting name

* Update index.tsx

* better showOffline

* Update index.js

* Update index.tsx

* Update index.tsx

* extract type text func

* Update cardDetailUtils.ts

* Revert "Update cardDetailUtils.ts"

This reverts commit d16d766.

* fix fall back offline payment text

* remove promo code for offline payment

* add background to offline svg

* Update offline.svg

* padding fixes

* svg

* add defaultPaymentMethodNotAllowedText for offline

* remove unused
* Update index.js

* Update index.js

* more generic

* test id

---------

Co-authored-by: EliFrankel <[email protected]>
* hide price (#791)

* Update cardDetailUtils.ts

* Revert "Update cardDetailUtils.ts"

This reverts commit d16d766.

* hide price

* move to settings context

* remove loadShowPrice from context

* Revert "remove loadShowPrice from context"

This reverts commit d94850e.

* id

* testID

---------

Co-authored-by: Omer Gery <[email protected]>
Co-authored-by: EliFrankel <[email protected]>

* change default to false (#795)

* add stripe key to debug mode

* change default to false

---------

Co-authored-by: Omer Gery <[email protected]>

---------

Co-authored-by: Omer Gery <[email protected]>
Co-authored-by: EliFrankel <[email protected]>
* hide actual text when hide price

* remove estimatedText
@OmerGery OmerGery closed this Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.